Replace QueryPatternSearch with SearchManager across services#7
Merged
Conversation
- Removed the `QueryPatternSearch` class and all associated logic. - Refactored `ContextBuilder` and tools to use the new `SearchManager` for query pattern and learning searches. - Added support for custom knowledge indexes in the `Context` model. - Updated `SemanticModelLoader` to load table metadata exclusively from the database, removing file-based support. - Enhanced `SearchKnowledgeTool` to dynamically handle multiple indexes and improve formatting of results.
- Deleted `runtimeSchema` from `Context`, `ContextBuilder`, and tests. - Refactored context generation to rely on pre-defined semantic models and business rules. - Removed `SchemaIntrospector` and all runtime schema assembly logic. - Updated documentation to clarify that runtime schema is fetched on-demand using tools during the agent loop.
…lowchart - Added Mermaid diagram to README for visualizing the agent workflow. - Expanded explanations to cover the newly introduced six context layers. - Updated "Context Assembly" sections to reflect distinctions between layers sourced from knowledge bases and those generated on-demand.
- Introduced optional integration with Prism Relay for registering tools from Model Context Protocol (MCP) servers. - Added tests to validate Relay integration, tool registration, and compatibility with custom tools. - Updated documentation to include setup instructions, configuration examples, and usage scenarios. - Enhanced `SqlAgentServiceProvider` to dynamically register tools from MCP servers if `prism-php/relay` is installed. - Modified `composer.json` to suggest the `prism-php/relay` package for MCP relay functionality.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
QueryPatternSearchclass and all associated logic.ContextBuilderand tools to use the newSearchManagerfor query pattern and learning searches.Contextmodel.SemanticModelLoaderto load table metadata exclusively from the database, removing file-based support.SearchKnowledgeToolto dynamically handle multiple indexes and improve formatting of results.